Perfect. While you are at it, would you mind renaming ids_... to inds_...? I don't care to much for abbreviations where random letters have been omitted 😄 .

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.90%. Comparing base (d68876b) to head (357cb46).
Report is 1 commits behind head on master.

@@           Coverage Diff           @@
##           master     #254   +/-   ##
=======================================
  Coverage   82.90%   82.90%           
=======================================
  Files          44       44           
  Lines        5749     5751    +2     
=======================================
+ Hits         4766     4768    +2     
  Misses        983      983           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Strange errors; would this be a result of the Strided.jl changes, where copyto! no longer goes via Strided.jl. It seems some paths of Strided.mul! go to copyto!, which I should also have replaced with copy! statements. I will fix this asap.

Looking at the stacktraces I think this is indeed the issue: rmul!(stridedarray, 1) goes through copyto! which checks for aliasing.

Strided.jl v2.3.2 will be tagged/registered soon, so we can then rerun the tests.